Skip to content

Conversation

JSpon
Copy link
Contributor

@JSpon JSpon commented Nov 6, 2024

Several data sources and resources have project support. This is to add it to more data sources and resources.

@CodeBleu CodeBleu added the enhancement New feature or request label Dec 18, 2024
@CodeBleu CodeBleu added this to the v0.6.0 milestone Dec 18, 2024
@CodeBleu
Copy link
Collaborator

@JSpon Thanks for your contribution. Can you please add tests for this? It's also good to follow this once you've added your tests as this can help catch things before the github workflow runs it.

@weizhouapache
Copy link
Member

there are several issues with this PR

I created a PR for vpc in project: #209

Schema: map[string]*schema.Schema{
"filter": dataSourceFiltersSchema(),

"projectid": {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is already a field project. Can't we use that instead?

Schema: map[string]*schema.Schema{
"filter": dataSourceFiltersSchema(),

"projectid": {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is already a field project. Can't we use that instead?

@kiranchavala
Copy link
Collaborator

@JSpon Could you please resolve the review comments

@kiranchavala
Copy link
Collaborator

@JSpon could you please let us know if you want the PR in the upcoming release?

Currently, we need the PR to pass the tests and the conflicts must be resolved inorder to merge it

@kiranchavala
Copy link
Collaborator

@JSpon closing this as pr #167 solves the use case of mentioning using the datasource for project

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants